Skip to content

Import and decompile ten MAGIC overlays - #326

Open
khasinski wants to merge 5 commits into
Xeeynamo:mainfrom
khasinski:magic-ten-overlays
Open

khasinski wants to merge 5 commits into
Xeeynamo:mainfrom
khasinski:magic-ten-overlays

Conversation

@khasinski

Copy link
Copy Markdown
Contributor

Imports and decompiles ten MAGIC overlays: GATTAI, JINRAI, SMKBALL, THREAT, TEARS, ECHOES, MAGEND, HAKAI, BOOST, and MINIMAM.

@Xeeynamo Xeeynamo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few changes needed:

  • All D_80062D98 checks must do an early return, instead of nesting the whole function inside an if
  • Some while loops I'd suggest attempting to convert as for loops
  • need a rebase

I'd be good to also import data and assets out of those magic overlays, not only getting the code decompiled. Otherwise they are not really 100% complete.

Comment thread src/magic/echoes.c Outdated
Comment thread src/magic/hakai.c Outdated
Comment thread src/magic/minimam.c Outdated
Comment thread assets/us/magic/boost_model.databin.bin Outdated

@Xeeynamo Xeeynamo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting a few more changes. I know I mentioned importing data and assets for the magic files. But I think it would have been better to have done that in a separate PR, so it's much easier for me to review. We do not want to have PRs that keep growing in scope review after review.

Comment thread src/magic/tears.c Outdated
Comment thread src/magic/minimam_data.c Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants